Skip to content

Conversation

@erikd
Copy link
Contributor

@erikd erikd commented Jan 11, 2026

Description

Anybody that wants to build Haskell code on top of db-sync and write Hasql queries will need at least the DbM type exposed and probably other stuff in that module.

Checklist

  • Commit sequence broadly makes sense
  • Commits have useful messages
  • New tests are added if needed and existing tests are updated
  • Any changes are noted in the changelog
  • Code is formatted with fourmolu on version 0.17.0.0 (which can be run with scripts/fourmolize.sh)
  • Self-reviewed the diff

Migrations

  • The pr causes a breaking change of type a,b or c
  • If there is a breaking change, the pr includes a database migration and/or a fix process for old values, so that upgrade is possible
  • Resyncing and running the migrations provided will result in the same database semantically

If there is a breaking change, especially a big one, please add a justification here. Please elaborate
more what the migration achieves, what it cannot achieve or why a migration is not possible.

Anybody that wants to build Haskell code on top of `db-sync` and
write `Hasql` queries will need at least the `DbM` type exposed
and probably other stuff in that module.
@erikd erikd requested a review from a team as a code owner January 11, 2026 02:18
@erikd erikd force-pushed the erikd/expose-types branch from b10d194 to 250f371 Compare January 11, 2026 05:00
Copy link
Contributor

@Cmdv Cmdv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch thank you 👍

@kderme
Copy link
Contributor

kderme commented Jan 12, 2026

Cardano.DB.Types is reexported through Cardano.DB, as most things in cardano-db. This should give access.

@erikd
Copy link
Contributor Author

erikd commented Jan 12, 2026

Cardano.DB.Types is reexported through Cardano.DB, as most things in cardano-db. This should give access.

Yes, you are correct. Thanks.

@erikd erikd closed this Jan 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants